Skip to content

Queues - Sahana Murthy - Task List #37

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Queues - Sahana Murthy - Task List #37

wants to merge 10 commits into from

Conversation

@sahanamurthy
Copy link

@sahanamurthy sahanamurthy commented Mar 27, 2017

Task List

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Describe in your own words what the Model is doing in Rails The Model speaks with the database (Active Record).
Describe in your own words what the Controller is doing in Rails The Controller receives requests from the browser and passes them on to the Model and View as appropriate, and also sends a response to the browser.
Describe in your own words what the View is doing in Rails The View handles the user interface. It displays text/images/styling for the webpage.
What is the purpose of using strong params? (i.e. the params method in the controller) The purpose of using strong params is to prevent outside parties from altering the objects of the program.
How are Rails migrations related to Rails models? Rails migrations are changes made to a Rails model.
Describe one area of Rails that are still unclear on All of it. What is this. What's happening.

Sahana Murthy added 10 commits March 20, 2017 14:28
Created rails files and tasks controller
d18a6b1
Created tasks controller class
6ff1f78
Created new tasks controller
84de383
Created the tasks index file
c6c4163
Created @tasks hash
9b5bf8d
Basic tasks list showing on screen
3978c30
Completed Wave 1, Wave 2, Wave 3. For those of you who are monitoring…
8d12d1f 
… my git hygiene, I am so sorry I haven't been pushing and committing more. I didn't know that we would be submitting this assignment. I would have done a much better job of committing otherwise. I am hoping to commit at a more acceptable rate going forward. Excited for Wave 4. Need to work on CSS and webpage display.
Tutoring session code
530e528
All requirements met and working
35c6139
Project completed.
aa7df12
@PilgrimMemoirs
Copy link

PilgrimMemoirs commented Apr 4, 2017

Task List

What We're Looking For

Feature Feedback
Baseline
Appropriate Git Usage with no extraneous files checked in Well done with no extra files - could commit more frequently
Answered comprehension questions Mostly Good - The model is a class defined by a table in the database, it also handles business logic, if there is any. Migrations also define the structure of your applications's database (what tables there are, their columns and the datatypes)
Successfully handles: Index, Show Well Done
Successfully handles: New, Create Well Done
Successfully handles: Edit, Update Well done
Successfully handles: Destroy, Task Complete
Routes follow RESTful conventions Well Done - Remember to set a root path.
Uses named routes (like _path) Well Done
Overall
Could use a partial for the new/edit form - since they're the same.
Great work - Nice presentation of tasks in index, with having the completed tasks show in a different list

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sahanamurthy @PilgrimMemoirs